Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change guest cpu topology checking method #5736

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

liang-cong-red-hat
Copy link
Contributor

@liang-cong-red-hat liang-cong-red-hat commented Jun 27, 2024

Change cpu topology checking method, using get_cpu_topology_in_vm() instead. And this commit is depends on avocado-vt pr: avocado-framework/avocado-vt#4038

results:
(1/2) type_specific.io-github-autotest-libvirt.guest_numa_topology.numa_topology_with_cpu_topology.vcpu_with_order: STARTED
(1/2) type_specific.io-github-autotest-libvirt.guest_numa_topology.numa_topology_with_cpu_topology.vcpu_with_order: PASS (66.80 s)
(2/2) type_specific.io-github-autotest-libvirt.guest_numa_topology.numa_topology_with_cpu_topology.vcpu_without_order: STARTED
(2/2) type_specific.io-github-autotest-libvirt.guest_numa_topology.numa_topology_with_cpu_topology.vcpu_without_order: PASS (66.78 s)

@liang-cong-red-hat liang-cong-red-hat force-pushed the cpu_topology_change_verification branch from 8c07a5a to 9167120 Compare June 27, 2024 08:48
@liang-cong-red-hat
Copy link
Contributor Author

@dzhengfy pls help review, thx

@liang-cong-red-hat liang-cong-red-hat force-pushed the cpu_topology_change_verification branch from 9167120 to 8c1a359 Compare December 10, 2024 06:26
@liang-cong-red-hat liang-cong-red-hat changed the title Change guest cpu dies checking methods Change guest cpu topology checking methods Dec 10, 2024
@liang-cong-red-hat liang-cong-red-hat changed the title Change guest cpu topology checking methods Change guest cpu topology checking method Dec 10, 2024
@dzhengfy dzhengfy added the depend on The PR has dependency on other PRs label Dec 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
depend on The PR has dependency on other PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants